Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support folder other than INBOX #921

Closed
wants to merge 1 commit into from
Closed

Support folder other than INBOX #921

wants to merge 1 commit into from

Conversation

herver
Copy link

@herver herver commented Jan 9, 2014

This small patch add a folder parameter that defaults to "INBOX" to the IMAP input

Add folder parameter that defaults to "INBOX"
@electrical
Copy link
Contributor

Hi,

Could you do step 2 of our contributing.md ?
Thanks!

@electrical
Copy link
Contributor

@herver Any update on this? Would like to merge it.

Cheers.

@herver
Copy link
Author

herver commented Mar 20, 2014

@electrical Step 2 (and 3) done ! Sorry for the delay !

@jrgns
Copy link

jrgns commented Aug 7, 2014

👍 Small but usefull change!

@jordansissel
Copy link
Contributor

Reviewed the patch. LGTM.

Thank you for helping make Logstash more awesome :)

@elasticsearch-release
Copy link

Can one of the admins verify this patch?

@wiibaa
Copy link
Contributor

wiibaa commented Dec 2, 2014

CanBeClosed. Moved to logstash-plugins/logstash-input-imap#2

@jordansissel
Copy link
Contributor

@herver did you recently change your github username? The git commit says the author is munsternet.

(view it here: https://github.com/elasticsearch/logstash/pull/921.patch)

@jordansissel
Copy link
Contributor

@herver the reason I ask is the git commit log has an author who's email is not listed in our "signed CLA".

jordansissel pushed a commit to logstash-plugins/logstash-input-imap that referenced this pull request Dec 2, 2014
@jordansissel
Copy link
Contributor

Migrated as above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants